Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix tidy with maintain_stored_certificate_counts == publish_stored_certificate_count_metrics == false #20664

Merged
merged 3 commits into from
May 19, 2023

Conversation

cipherboy
Copy link
Contributor

The logic around the check to set both to false was wrong, and should be validated independently.

Additionally, these fields should only exist on auto-tidy and not on the manual tidy endpoint.

…rtificate_count_metrics == false

The logic around the check to set both to false was wrong, and should
be validated independently.

Additionally, these fields should only exist on auto-tidy and not on the
manual tidy endpoint.

Signed-off-by: Alexander Scheel <[email protected]>
@cipherboy cipherboy added bug Used to indicate a potential bug secret/pki backport/1.12.x labels May 18, 2023
@cipherboy cipherboy added this to the 1.12.7 milestone May 18, 2023
@cipherboy cipherboy enabled auto-merge (squash) May 19, 2023 11:38
Signed-off-by: Alexander Scheel <[email protected]>
@cipherboy cipherboy merged commit 7144523 into main May 19, 2023
@cipherboy cipherboy deleted the cipherboy-fix-bug-setting-both-fields-false branch May 19, 2023 12:09
cipherboy added a commit that referenced this pull request May 19, 2023
Fix tidy with maintain_stored_certificate_counts == publish_stored_certificate_count_metrics == false (#20664)

* Fix tidy with maintain_stored_certificate_counts == publish_stored_certificate_count_metrics == false

The logic around the check to set both to false was wrong, and should
be validated independently.

Additionally, these fields should only exist on auto-tidy and not on the
manual tidy endpoint.

Signed-off-by: Alexander Scheel <[email protected]>

* Update builtin/logical/pki/path_tidy.go

Co-authored-by: claire bontempo <[email protected]>

---------

Signed-off-by: Alexander Scheel <[email protected]>
Co-authored-by: claire bontempo <[email protected]>

Signed-off-by: Alexander Scheel <[email protected]>
@cipherboy cipherboy modified the milestones: 1.12.7, 1.13.3 May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to indicate a potential bug secret/pki
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants